🙈 The PR is closed and the preview is expired.

On my screen (which is large, but not redonculously large), the sizing seems a bit off, relative to the text in the other bands:

The screen cap doesn't really capture it (since the screen cap is in a smaller space), but the logo is huge, and takes up almost the whole of the large window.

Also, I wonder if the text next to the logo would look better if it was vertically centred, so it was less out of alignment with the text in the logo?

If I shrink the window and change the aspect ratio, the alignment looks "right", and the size of the logo seems sensible:

So I wonder if we just need an upper cap on the size of the logo on wide screens, so it doesn't take up the whole vertical space?

@holly-cummins - We use a 12 column grid and that version it was a 7/5 split of logo to content. I've reversed it to 5/7 with my last commit.

My screen cap at 1920...

It looks great and I'm eager to merge it - would it be ok if I pressed the big green button?

I do have some minor nitpicks but I think I'd prefer to let @insectengine to choose if he'd want to handle them as a follow up or not at all :)

Here is my screenshot (just the bottom bit):

Two things stand out:

• the commonhaus logo in the footer seems rather big - perhaps it should be of similar height of the Quarkus logo which is in the center/top of the picture?
• the "Sponsored by" lettering seems to drift a bit far from the Red Hat logo on such a display

Both of these look totally fine on mobile though. I appreciate my display is large 3360x1440 - on the other hand I expect most developers to have large screens, often even larger.

Neither seem blocking so I'd rather merge and polish later though. It's also still undecided if we should keep the "Sponsored by Red Hat" altogether so wouldn't want you to spend too many cycles on it, if it's not a trivial change.

It looks great and I'm eager to merge it - would it be ok if I pressed the big green button?

Actually let's just do it, @insectengine I'll leave it up to you to decide if my nitpicks are worthy of a follow up. Thanks a lot!

Thanks @insectengine, @holly-cummins and @Sanne. The same should be used in the other CommonHaus project that moved at the same time as Quarkus (SmallRye, Qbic)

\CC @radcortez @dmlloyd @jponge

Thanks I'll transition that to the Mutiny projects, and future releases will reflect that change.

Good point, but bear in mind that there's the possibility that @insectengine will want to apply some more tweaks - perhaps give it some days for things to settle, so to avoid a multitude of patches.

@Sanne - Think this issue will require a change of how this is built as columns based layout will force this space. We can limit the size of the CF logo, but that will just emphasize the spacing problem. I'll have to refactor this a bit #bigscreenproblems

@jponge @radcortez @dmlloyd - hold off on this until I get this sorted.